-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
INTERNAL PR VidosResolver example #1
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this, this is a really good solution. Couple of small changes and let's get the other pull request merged.
verifier-integration/js/index.js
Outdated
const VIDOS_RESOLVER_URL = undefined; // 'https://ugly-aqua-hummingbird-120.resolver.service.eu.vidos.local'; | ||
const VIDOS_API_KEY = undefined; // '24b500499e7c2f0a8634831d1b9894b74d656d3e7cd6de238d706c20a5db6d41'; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like some debug lines here
verifier-integration/js/index.js
Outdated
@@ -85,9 +89,10 @@ async function callback(req, res) { | |||
ethURL, | |||
contractAddress | |||
); | |||
const vidosResolver = VIDOS_RESOLVER_URL && VIDOS_API_KEY ? new VidosResolver(VIDOS_RESOLVER_URL, VIDOS_API_KEY) : undefined; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need a way to get the config inserted into these environment variables
}, | ||
}); | ||
const result = await response.json(); | ||
// console.log("root result", JSON.stringify(result, null, 2)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Presumably a debugging statement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, made a small change to remove specific endpoint
VidosResolver Implementation: mailchain/js-iden3-auth#1
To get this example working while there is no js-iden3-auth lib published that includes
VidosResolver
js-iden3-auth
donpm install
,npm run build
andnpm link
vidos-resolver-example
polygonid-tutorial-examples
, cd intoverifier-integration/js
package,npm install
andnpm link @iden3/js-iden3-auth
.env
fileNODE_TLS_REJECT_UNAUTHORIZED="0" node .
otherwise justnode .